Skip to content

Fix FEX access untagged fault parsing - #482

Merged
monrog2 merged 1 commit into
v4.3.1-devfrom
codex/issue-481-access-untagged-fex-dn
Oct 27, 2026
Merged

monrog2 merged 1 commit into
v4.3.1-devfrom
codex/issue-481-access-untagged-fex-dn

Conversation

@monrog2

@monrog2 monrog2 commented Oct 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • parse both APIC-documented fvStPathAtt and fvExtStPathAtt fault DN forms
  • preserve the FEX ID in the existing Port result column
  • fix the unformatted fallback so an unknown DN reports a failure instead of raising TypeError
  • add scrubbed regression fixtures for standard, FEX, and unrecognized attachment paths

Testing

  • python3 -m pytest -q tests/checks/access_untagged_check/test_access_untagged_check.py — 4 passed
  • python3 -m pytest -q — 1,465 passed
  • Python 2.7 is not installed locally; GitHub CI will validate that supported runtime

Closes #481

@monrog2 monrog2 linked an issue Oct 14, 2026 that may be closed by this pull request
@monrog2

monrog2 commented Oct 14, 2026

Copy link
Copy Markdown
Collaborator Author

Integration Test Summary

A focused GitLab integration run was completed for access_untagged_check against commit df13276464d4d8990b0525c1d5816f38fbd0ad46.

  • Python 2.7 and Python 3.8 unit tests passed.
  • All seven integration scenarios executed the intended focused check.
  • The original append() TypeError did not recur.
  • No validator ERROR !! results were observed.
  • Each scenario correctly processed and displayed one legitimate F0467 finding as FAIL - OUTAGE WARNING!!, including the parsed interface information and recommended action.

The pipeline’s overall failure was caused by test-environment availability:

  • One required fabric consistently failed its initial APIC query because its local session-token mount was unavailable.
  • Other required fabrics encountered intermittent SSH authentication or socket-timeout failures.
  • Nine optional fabrics per scenario reported connection warnings; the integration runner explicitly excluded these from validator errors.

No customer names, fabric identifiers, addresses, tenants, application profiles, or EPG names are included in this summary. The focused validation indicates that the code fix works as intended.

@monrog2
monrog2 changed the base branch from v4.3.0-dev to v4.3.1-dev October 27, 2026 13:18
@monrog2
monrog2 merged commit f947669 into v4.3.1-dev Oct 27, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Access (Untagged) Port Config check crashes on FEX attachment fault DN

1 participant